Skip to content

fix: recover bump files for release notes and fix git tag push auth#73

Merged
theoephraim merged 2 commits into
mainfrom
fix/release-tags-and-release-notes
May 13, 2026
Merged

fix: recover bump files for release notes and fix git tag push auth#73
theoephraim merged 2 commits into
mainfrom
fix/release-tags-and-release-notes

Conversation

@theoephraim
Copy link
Copy Markdown
Member

Summary

  • Fix git tag push failure: withGitToken was embedding the token in the remote URL (x-access-token:TOKEN@github.com/...), but newer git credential managers on CI runners intercept the connection and ignore URL-embedded credentials, causing git push --tags to fail with "could not read Username". Now uses http.<server>/.extraheader = Authorization: bearer TOKEN instead — the same mechanism actions/checkout uses, which takes priority over credential managers.
  • Remove redundant git push in pushWithTags: The commit push before git push --tags is unnecessary in the publish flow (no new commits to push) and was the command that was actually failing first.
  • Recover deleted bump files for GitHub release notes: In the post-version-PR-merge publish flow, bump files are already deleted. ciReleaseCommand now recovers them from the version commit (git diff HEAD~1 HEAD --diff-filter=D) and passes them to publishCommand, so GitHub release bodies are populated with proper changelog content.
  • Add extractChangelogEntry helper: Extracts a specific version's entry from a CHANGELOG.md string (for use in release body generation).

Test plan

  • Merge a version PR and verify git tags are pushed successfully in CI
  • Verify GitHub releases have proper release notes (not empty bodies)
  • Verify the recoverDeletedBumpFiles correctly reads bump file content from HEAD~1

- Recover deleted bump files from version commit (HEAD~1) so GitHub release
  bodies are populated with changelog content in the post-merge publish flow
- Fix git tag push failure: switch from URL-embedded token to http.extraheader
  (same mechanism as actions/checkout), which takes priority over credential
  managers on CI runners
- Remove redundant `git push` before `git push --tags` in pushWithTags — in
  the publish flow there are no new commits to push
- Add extractChangelogEntry helper to extract changelog entries from CHANGELOG.md
@github-actions
Copy link
Copy Markdown

bumpy-frog

The changes in this PR will be included in the next version bump.

patch Patch releases

  • @varlock/bumpy 1.8.0 → 1.8.1

Bump files in this PR

Click here if you want to add another bump file to this PR


This comment is maintained by bumpy.

@theoephraim theoephraim merged commit 84413fc into main May 13, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant